Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add button that finds missing OCR captions #32

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

thiswillbeyourgithub
Copy link

@thiswillbeyourgithub thiswillbeyourgithub commented Nov 18, 2021

Hi !

I decided to add a new menu button that allows to find the missing captions in the collection. Hope you like it!

You can find reference to this in the issue #17

Have a nice day and good luck on your studies!

@thiswillbeyourgithub
Copy link
Author

I just realized that some users use the OCR in a dedicated field instead of as html code.

Do you think it would be good enough to have this button appear only if the setting is set to "tooltip" ?

@cfculhane
Copy link
Owner

Hi there, thanks for this - I'm literally in the middle of exam period, but will get this merged in a week or so. Thanks for your contribution @thiswillbeyourgithub !

@thiswillbeyourgithub
Copy link
Author

thiswillbeyourgithub commented Nov 19, 2021

Good luck on your exams :) The world needs more doctors that know how to code >_< !

Btw, I'm still on anki 2.1.44 so I don't really know if it works in new versions.

edit : you could also use this to automatically rerun OCR on them instead of adding a tag, I could code it it you want me to.

@thiswillbeyourgithub
Copy link
Author

thiswillbeyourgithub commented Feb 4, 2022

I added a new commit for a new button :)

I tested it and it seems to work fine on anki 2.1.49

Also: less false positive because .gif and .svg are ignored :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants